Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Msbuild tasks. #5

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Msbuild tasks. #5

wants to merge 1 commit into from

Conversation

Kuinox
Copy link
Owner

@Kuinox Kuinox commented Aug 25, 2024

Fixes #2.

@BinToss, I think I will create a regular Msbuild task that do nothing, and have another package that magically make the nuget reproducible, so if anyone want to use the task directly, they can.

@Kuinox
Copy link
Owner Author

Kuinox commented Aug 27, 2024

I finish wrapping up https://github.com/Kuinox/Myna and i'll get back to this PR and then release the packages.

@BinToss
Copy link
Contributor

BinToss commented Aug 27, 2024

Take your time. I'll be busy with other things for a week or two.

/// <summary>
/// The last write time to sets. Default to 2000-01-01.
/// </summary>
public DateTime LastWriteTime { get; set; } = new DateTime(2000, 1, 1);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know which PR gets in first, but this creates a semantic merge conflict between them. SOURCE_DATE_EPOCH should clearly be used in the initializer here too.

Copy link
Contributor

@BinToss BinToss Oct 20, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://learn.microsoft.com/en-us/dotnet/api/system.datetime.unixepoch?view=net6.0
For later reference, UnixEpoch is pre-defined at this location in the .NET 6 API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Kuinox.NupkgDeterministicator.MSBuild package
3 participants